Skip to content

musl: Make rustc-dep-of-std imply musl_v1_2 - #5447

Open
Gelbpunkt wants to merge 1 commit into
rust-lang:mainfrom
Gelbpunkt:rust-dep-of-std-musl-1-2
Open

Gelbpunkt wants to merge 1 commit into
rust-lang:mainfrom
Gelbpunkt:rust-dep-of-std-musl-1-2

Conversation

@Gelbpunkt

Copy link
Copy Markdown
Contributor

The musl baseline for Rust is 1.2.5 since 1.93.0. Therefore, it is safe to make rustc-dep-of-std imply musl_v1_2, and this lets us start using statx from libc in std.

The musl baseline for Rust is 1.2.5 since 1.93.0. Therefore, it is safe
to make rustc-dep-of-std imply musl_v1_2, and this lets us start using
statx from libc in std.

@tgross35 tgross35 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The change here looks fine to me, though a comment in Cargo.toml would be helpeful. However, I'd rather not merge this until there's a rust-lang/rust PR setting this cfg via bootstrap and doing try jobs on our musl targets. That won't be the permanent of course, but I would be better to find anything broken and fix it without needing to ping pong between libc releases and rustc PRs.

View changes since this review

@rustbot

rustbot commented Aug 30, 2026

Copy link
Copy Markdown
Collaborator

Reminder, once the PR becomes ready for a review, use @rustbot ready.

@tgross35 tgross35 added the stable-nominated This PR should be considered for cherry-pick to libc's stable release branch label Aug 31, 2026
@rustbot

rustbot commented Sep 7, 2026

Copy link
Copy Markdown
Collaborator

☔ The latest upstream changes (possibly #5508) made this pull request unmergeable. Please resolve the merge conflicts.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

O-linux O-linux-like O-musl O-unix S-waiting-on-author stable-nominated This PR should be considered for cherry-pick to libc's stable release branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants